-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update DWN SDK #7
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, looks good! Just a couple of comments from me. Holding final approval until those last dwn-sdk-js
changes get in.
Codecov Report
@@ Coverage Diff @@
## main #7 +/- ##
==========================================
+ Coverage 77.04% 80.02% +2.97%
==========================================
Files 10 10
Lines 806 956 +150
Branches 120 140 +20
==========================================
+ Hits 621 765 +144
- Misses 185 191 +6 |
@amika-sq version was bumped on sdk, ready for final review. |
Updating the DWN SDK to the latest version.
This Includes:
encodedData
stored within theMessageStore
below a certain thresholdMessageStore
interface now accepts multiple Filters to create OR conditions between filters.datePublished
, add the propertypublished
set to true within the filter.Relevent
dwn-sdk-js
PRs:decentralized-identity/dwn-sdk-js#478
decentralized-identity/dwn-sdk-js#456